Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

chore(reference): use API Extractor for TypeScript rollup #4538

Merged
merged 2 commits into from
Nov 22, 2024

Conversation

glowcloud
Copy link
Contributor

Refs #4382

@glowcloud glowcloud added enhancement New feature or request TypeScript TypeScript related work labels Nov 21, 2024
@glowcloud glowcloud self-assigned this Nov 21, 2024
@@ -31,6 +32,13 @@ export type { BundleStrategyOptions } from './bundle/strategies/BundleStrategy.t

export { default as options } from './options/index.ts';
export { merge as mergeOptions } from './options/util.ts';
export type {
ReferenceOptions as ApiDOMReferenceOptions,
Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Went with the ApiDOM prefix for these exports as ReferenceOptions has to be renamed due to another export:

export type { ReferenceOptions } from './Reference.ts';

Not sure what else we could go for, except for perhaps dropping the Reference part from all of them

Suggested change
ReferenceOptions as ApiDOMReferenceOptions,
ReferenceOptions as Options,
ReferenceBundleOptions as BundleOptions,
...

@glowcloud glowcloud merged commit 3d5243f into main Nov 22, 2024
8 checks passed
@glowcloud glowcloud deleted the 4382-apidom-reference branch November 22, 2024 11:45
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request TypeScript TypeScript related work
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants